Skip to content

feat(mixin): add mixin recording feature#13

Open
SettingDust wants to merge 22 commits into
terrarium-earth:mainfrom
SettingDust:feature/build-time-mixin
Open

feat(mixin): add mixin recording feature#13
SettingDust wants to merge 22 commits into
terrarium-earth:mainfrom
SettingDust:feature/build-time-mixin

Conversation

@SettingDust

Copy link
Copy Markdown
Contributor

Depends on #12

@SettingDust SettingDust marked this pull request as draft April 25, 2025 02:55
@SettingDust

SettingDust commented Apr 25, 2025

Copy link
Copy Markdown
Contributor Author

Build time mixin almost done. But has fatal issue with multiple targets. Since the mixin isn't designed for compile/build time and applying multiple times. Mixin extras isn't supporting initialize multiple times at the same time.
Need to isolate the class loaders for different targets, as don't use the same mixin classes for times. And we can support different mixin variants in this way. But it's... pAin 😢
Almost a rewrite for the codev mixin module.

Can someone help me to implement it? I can provide some example and advice, since I've implemented a new isolate mixin service targeting add-ons from a forge mod but an add-on loader.

I'm going to implement the include extract first. If no one. I'll back after it.

…mixin

# Conflicts:
#	gradle.properties
#	minecraft-codev-includes/src/main/kotlin/net/msrandom/minecraftcodev/includes/ExtractIncludes.kt
…mixin

# Conflicts:
#	minecraft-codev-forge/src/main/kotlin/net/msrandom/minecraftcodev/forge/task/ResolvePatchedMinecraft.kt
…mixin

# Conflicts:
#	minecraft-codev-forge/src/main/kotlin/net/msrandom/minecraftcodev/forge/task/ResolvePatchedMinecraft.kt
#	minecraft-codev-remapper/src/main/kotlin/net/msrandom/minecraftcodev/remapper/JarRemapper.kt
@SettingDust SettingDust marked this pull request as ready for review March 29, 2026 09:02
@SettingDust SettingDust force-pushed the feature/build-time-mixin branch from a4a2f42 to 434db97 Compare March 29, 2026 09:05
@SettingDust SettingDust force-pushed the feature/build-time-mixin branch from 434db97 to a45f6f1 Compare March 29, 2026 09:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant